Skip to content

Conversation

lumirlumir
Copy link
Collaborator

@lumirlumir lumirlumir commented Sep 21, 2024

안녕하세요😊

Windows 환경에서 yarn을 통해 husky를 사용할 때, stdin is not a tty 에러가 발생하는 이슈가 있어 해당 버그를 수정합니다.

스크린샷 2024-09-21 151957

husky 공식 문서 및 issue에 해결 방법이 있어 그대로 적용하였습니다.

image

스크린샷 2024-09-21 154911

Progress

@vercel
Copy link

vercel bot commented Sep 21, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ko-react-dev 🔄 Building (Inspect) Visit Preview 💬 Add feedback Sep 23, 2024 6:50am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
ko-legacy-reactjs-org ⬜️ Ignored (Inspect) Visit Preview Sep 23, 2024 6:50am

@github-actions
Copy link

github-actions bot commented Sep 21, 2024

Size changes

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@lumirlumir lumirlumir marked this pull request as ready for review September 21, 2024 08:05
@lumirlumir lumirlumir changed the title fix: stdin is not a tty error in husky fix: stdin is not a tty error in husky Sep 21, 2024
hg-pyun
hg-pyun previously approved these changes Sep 22, 2024
Copy link
Member

@hg-pyun hg-pyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lumirlumir lumirlumir merged commit 8ad68ca into reactjs:main Sep 24, 2024
5 checks passed
@highspirit7
Copy link
Contributor

@lumirlumir 어디서 이 얘기를 드려야 할지 몰라서 여기에 남깁니다. 저는 mac os를 사용하고 있고, 오늘 이 레포지토리 포크, 클론 후 커밋을 남기려고 할때 아래와 같은 에러가 발생해서 커밋을 할 수가 없었습니다. 제가 확인한 바로는 이 풀리퀘스트로 추가된 common.sh 파일이 CRLF (Carriage Return + Line Feed) 개행문자를 사용하고 있어서 맥에서 실행하는데 생긴 문제였습니다. mac os는 LF(Line Feed) 개행문자를 기반이라서 LF로 변환해주는 도구를 사용한 뒤 커밋을 할 수는 있었습니다. 앞으로 맥이나 리눅스 유저는 이렇게 변환해서 커밋을 해야하는 것인지 어떻게 이 문제를 해결해야하는지 제 수준으로서는 판단할 수가 없어서 일단 이렇게 남겨봅니다.

Screenshot 2024-10-01 at 12 56 23 PM

@lumirlumir
Copy link
Collaborator Author

lumirlumir commented Oct 1, 2024

@lumirlumir 어디서 이 얘기를 드려야 할지 몰라서 여기에 남깁니다. 저는 mac os를 사용하고 있고, 오늘 이 레포지토리 포크, 클론 후 커밋을 남기려고 할때 아래와 같은 에러가 발생해서 커밋을 할 수가 없었습니다. 제가 확인한 바로는 이 풀리퀘스트로 추가된 common.sh 파일이 CRLF (Carriage Return + Line Feed) 개행문자를 사용하고 있어서 맥에서 실행하는데 생긴 문제였습니다. mac os는 LF(Line Feed) 개행문자를 기반이라서 LF로 변환해주는 도구를 사용한 뒤 커밋을 할 수는 있었습니다. 앞으로 맥이나 리눅스 유저는 이렇게 변환해서 커밋을 해야하는 것인지 어떻게 이 문제를 해결해야하는지 제 수준으로서는 판단할 수가 없어서 일단 이렇게 남겨봅니다.

Screenshot 2024-10-01 at 12 56 23 PM

@highspirit7 아 커밋 시 제 실수가 있었네요ㅠㅠ 오류 발견해주셔서 감사합니다🙇‍♂️

CRLF로 인해 발생한 오류가 맞습니다. 제가 Windows 유저다 보니 커밋 시 LF로 변환해서 올려야 하는데 실수로 CRLF로 올려버렸네요🥲🥲

원래 EditorConfig 같은 도구를 이용해서 모든 파일에 LF를 강제하는 것이 일반적입니다. 하지만, 현재 레포지토리에는 그런 툴이 존재하지 않아 문제가 발생했습니다.

해당 내용 수정 보완해서 PR 올리도록 할테니, 나중에 해당 내용 pull 받으시면 문제 없으실겁니다.

혹여, 현재 추가적으로 올릴 커밋 있으시다면 번거로우시더라도 (PR 반영되기 전까지) 잠깐만 직접 LF로 변환해서 사용해주시면 될 것 같습니다.

@lumirlumir lumirlumir deleted the fix-husky-stdin-is-not-a-tty branch October 1, 2024 13:03
@highspirit7
Copy link
Contributor

@lumirlumir 빠른 답변 감사합니다! 그런데 LF로 강제되면 윈도우에서 개발하실 때 생기는 문제는 없나요? 제가 윈도우로 개발해본 적이 없어서 질문드려봅니다.

@lumirlumir
Copy link
Collaborator Author

lumirlumir commented Oct 1, 2024

@highspirit7 VScode같은 코드 에디터를 이용하면 LF를 사용하던 CRLF를 사용하던 똑같이 개행을 처리해 주어서 크게 상관이 없습니다👍 그리고 개발할때는 LF가 사실상 표준이라 CRLF는 사용하는 경우가 드뭅니다...ㅠㅠ

@highspirit7
Copy link
Contributor

@lumirlumir 그렇군요. 윈도우에서 어떤 개행으로 되어있던 간에 실행하는데 문제는 없지만, 커밋 시 LF로 변환을 해주는 것이 필요한 거군요. 좋은 정보 알아갑니다. 감사합니다!

@lumirlumir
Copy link
Collaborator Author

lumirlumir commented Oct 1, 2024

@highspirit7 넵👍👍 #1068 PR과 곧 올릴 #1069 PR에 해당 내용 반영할 예정입니다. 나중에 한번 살펴봐 주세요~ 감사합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants